Skip to content

Remove wrappers and contracts per PR comments#2

Merged
celinval merged 1 commit intoverify-0000-unicodefrom
celinval-patch-1
Oct 28, 2024
Merged

Remove wrappers and contracts per PR comments#2
celinval merged 1 commit intoverify-0000-unicodefrom
celinval-patch-1

Conversation

@celinval
Copy link
Owner

Please add a description of your PR.
If this is a solution to an open challenge, please explain your solution.

Don't forget to check our book to ensure your solution satisfy the overall
requirements as well as the challenge success criteria.

Resolves #ISSUE-NUMBER

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

@celinval celinval merged commit a0aaf57 into verify-0000-unicode Oct 28, 2024
celinval pushed a commit that referenced this pull request Oct 29, 2025
…r=jieyouxu

Rehome 26 `tests/ui/issues/` tests to other subdirectories under `tests/ui/` [rust-lang#5 of Batch #2]

Part of rust-lang#133895

Methodology:

1. Refer to the previously written `tests/ui/SUMMARY.md`
2. Find an appropriate category for the test, using the original issue thread and the test contents.
3. Add the issue URL at the bottom (not at the top, as that would mess up stderr line numbers)
4. Rename the tests to make their purpose clearer

Inspired by the methodology that Kivooeo was using.

r? ```@jieyouxu```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant